Skip to content

Conversation

F0RRZZ
Copy link
Contributor

@F0RRZZ F0RRZZ commented Sep 14, 2025

@bedevere-app

This comment was marked as resolved.

Copy link
Contributor

@adorilson adorilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank for you PR, @F0RRZZ

I made some comments.

variables to their initial values:

* position: ``(0, 0)``
* heading: ``0`` (east)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe

  • heading: 0 (east if standard or world mode, north if logo mode)

Comment on lines 1399 to 1401
Delete the turtle's drawings from the screen. Move the turtle
to position ``(0, 0)``, set its heading to the default
(facing east in standard mode) and restore the following state
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these parentheses are unnecessary.

Maybe, you could say

set its heading to 0 and...

Lib/turtle.py Outdated
Comment on lines 2648 to 2654
Delete the turtle's drawings from the screen. Move the turtle
to position `(0, 0)`, set its heading to the default
(facing east in standard mode) and restore the following state
variables to their initial values:
- position: `(0, 0)`
- heading: `0` (east)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The same thing concerns heading here.

@F0RRZZ
Copy link
Contributor Author

F0RRZZ commented Sep 21, 2025

@adorilson Fixed, thank you for the suggestions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants